Port Scheduler library to Arduino Zero. #3958
Conversation
| @@ -64,11 +83,39 @@ static void __attribute__((naked)) __attribute__((noinline)) coopTaskStart(void) | |||
| static void __attribute__((naked)) __attribute__((noinline)) coopDoYield(CoopTask* curTask) { | |||
| asm ( | |||
| "mov r12, sp;" | |||
| "stmia r0, {r4-r12, lr};" | |||
| /* store low registers */ | |||
There was a problem hiding this comment.
Shall this be put inside ifdef too, so we can use the more efficient store instruction when available (without copying the high registers into the low registers)?
cmaglie
added
Architecture: SAM
cmaglie
added
Library: Scheduler
PeterVH
force-pushed
the
scheduler-zero
branch
from
edf3651 to
739c656
Compare
|
Yes indeed, that chunk of m0 specific code escaped my attention. B.t.w. I had big trouble testing this on my Due. Is there something seriously wrong with sam package 1.6.4? I could upload sketches to the Due (prog port) but could not run any simple led blinking program. Thought for a moment my Due was toasted. I reverted to 1.6.3 and everything was back ok. Anyway test passed on Due too. |
|
Should https://www.arduino.cc/en/Reference/Libraries list be updated to reflect Scheduler is "Due and Zero only", then? If it is, Zero board should be referenced too in first sentence of https://www.arduino.cc/en/Reference/Scheduler ... |
The library from the Due does not work as is on the Zero because it loads and stores the processor state using ldmia/stmia. On cortex m0, these instructions only work with the 'low' registers (r0-r7).
I made an implementation that loads the registers in the low registers first, and then moves them into the high regs.
Tested with MultipleBlink sketch.
Some feedback review on the code is desirable.
See forum